Skip to content

Respect existing Anthropic env vars in quickstart.sh - #6

Merged
stempeck merged 1 commit into
mainfrom
issues/5
May 6, 2026
Merged

Respect existing Anthropic env vars in quickstart.sh#6
stempeck merged 1 commit into
mainfrom
issues/5

Conversation

@stempeck

@stempeck stempeck commented May 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • Uses shell parameter expansion (${VAR:-default}) instead of unconditionally overwriting Anthropic environment variables in configure_shell()
  • Preserves any user-customized values already set in the shell profile

Test plan

  • Run quickstart.sh with no Anthropic vars set — defaults should apply
  • Run with pre-existing vars in profile — existing values should be preserved

🤖 Generated with Claude Code

@CLAassistant

CLAassistant commented May 6, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the quickstart shell-profile configuration to avoid clobbering user-provided Anthropic/Claude Code environment variables, while still supplying sane defaults for new setups.

Changes:

  • Switches configure_shell() model-related exports to use shell parameter expansion defaults (${VAR:-default}).
  • Preserves pre-set values by deferring the final value decision to shell startup time rather than hardcoding it in the generated block.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread quickstart.sh
@stempeck
stempeck merged commit 817e131 into main May 6, 2026
11 checks passed
@stempeck
stempeck deleted the issues/5 branch May 6, 2026 01:39
@stempeck stempeck linked an issue May 6, 2026 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update quickstart to retain anthropic profile settings

3 participants